Skip to content

Comments

fix openCV version in readme#291

Merged
atsju merged 2 commits intomasterfrom
JST/readme
Dec 3, 2025
Merged

fix openCV version in readme#291
atsju merged 2 commits intomasterfrom
JST/readme

Conversation

@atsju
Copy link
Collaborator

@atsju atsju commented Nov 30, 2025

No description provided.

@atsju atsju requested a review from gr5 November 30, 2025 06:21
@github-actions
Copy link

🚀 New build available for commit c2040b8
Download installer here

@atsju atsju marked this pull request as draft November 30, 2025 07:15
@atsju
Copy link
Collaborator Author

atsju commented Nov 30, 2025

draft as I need to check something regarding libopencv_flann4120.dll

@atsju atsju marked this pull request as ready for review November 30, 2025 08:08
@atsju
Copy link
Collaborator Author

atsju commented Nov 30, 2025

I just tried the lazy method from README file on a fresh computer. relative path for DLL were not working.

@github-actions
Copy link

🚀 New build available for commit 9a1e81f
Download installer here

@gr5
Copy link
Collaborator

gr5 commented Nov 30, 2025

I was using the pro file "as is" and it didn't seem to need a single edit and I was hoping I could build in QT6 with this pro file unedited from within qt creator. Adding the $$PWD I think will make it so I can no longer use the pro file as is. I will try to test this out today (build the way the readme file says and then build in qt creator).

@atsju
Copy link
Collaborator Author

atsju commented Nov 30, 2025

$$PWD expands to current .pro file location. It technically changes nothing but without it, build wasn't working today on the new computer.
This change should not affect you. It should not be required but at the same time it shouldn't harm anyone.

@atsju atsju merged commit 039bcc4 into master Dec 3, 2025
14 checks passed
@atsju atsju deleted the JST/readme branch December 3, 2025 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants